Skip to content

fix(react): avoid UTF-8 boundary panics - #167

Merged
sunerpy merged 2 commits into
mainfrom
fix/react-utf8-detection
Jul 24, 2026
Merged

fix(react): avoid UTF-8 boundary panics#167
sunerpy merged 2 commits into
mainfrom
fix/react-utf8-detection

Conversation

@sunerpy

@sunerpy sunerpy commented Jul 23, 2026

Copy link
Copy Markdown
Owner

Summary

  • make React framework lookahead windows contract to valid UTF-8 character boundaries
  • preserve the existing 500/400/300-byte extraction limits
  • add regression coverage for component, React Router, and data-router extraction paths

Verification

  • make ci
  • pre-push gate: fmt, clippy, workspace tests, and guardrail
  • targeted UTF-8 boundary regression tests
  • indexed an isolated 4,963-file oh-my-openagent checkout to an up-to-date graph without panic

Notes

  • one pre-push attempt observed the pre-existing timing-sensitive daemon_single_watcher_fires_once test reporting a zero-file sync; the test then passed three consecutive targeted runs, and the complete pre-push gate passed on retry

@codecov

codecov Bot commented Jul 23, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.

Impacted file tree graph

@@           Coverage Diff           @@
##             main     #167   +/-   ##
=======================================
  Coverage   95.26%   95.27%           
=======================================
  Files         124      124           
  Lines       61261    61312   +51     
=======================================
+ Hits        58359    58412   +53     
+ Misses       2902     2900    -2     
Files with missing lines Coverage Δ
crates/codegraph-resolve/src/frameworks/react.rs 94.70% <100.00%> (+0.31%) ⬆️

... and 2 files with indirect coverage changes

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@sunerpy
sunerpy merged commit ff0d504 into main Jul 24, 2026
7 checks passed
@sunerpy
sunerpy deleted the fix/react-utf8-detection branch July 24, 2026 01:13
This was referenced Jul 24, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant